Skip to content

gh-151943: Fix incorrect interface names in xml.sax.rst#151961

Open
manoj-marimuthu wants to merge 5 commits into
python:mainfrom
manoj-marimuthu:fix-sphinx-warnings-in-doc-lib-xml
Open

gh-151943: Fix incorrect interface names in xml.sax.rst#151961
manoj-marimuthu wants to merge 5 commits into
python:mainfrom
manoj-marimuthu:fix-sphinx-warnings-in-doc-lib-xml

Conversation

@manoj-marimuthu

@manoj-marimuthu manoj-marimuthu commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

The following PR addresses aims to remove the warnings from sphinx that are raised for xml.sax.rst. (total 3 warnings)

Warnings removed:

WARNING: py:class reference target not found: xml.sax.xmlreader.Attributes [ref.class]
WARNING: py:class reference target not found: xml.sax.xmlreader.AttributesNS [ref.class]
  • The above warnings were removed by replacing xml.sax.xmlreader.Attributes with xml.sax.xmlreader.AttributesImpl and
    xml.sax.xmlreader.AttributesNS with xml.sax.xmlreader.AttributesNSImpl.

Verification :

Warnings remaining:

WARNING: py:func reference target not found: create_parser [ref.func]
  • The above warning is due to the absence of documentation for create_parser function. We can either add documentation for it or remove the documentation's reference to create_parser. Waiting for suggestions.

@read-the-docs-community

read-the-docs-community Bot commented Jun 23, 2026

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #33420069 | 📁 Comparing b094606 against main (3f09a17)

  🔍 Preview build  

59 files changed · ± 58 modified · - 1 deleted

± Modified

- Deleted

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As for create_parser, please add a reference (.. _sax-parser-modules:) to the note at the top of the xml.sax.reader documentation. We can then expand that note.

Comment thread Doc/library/xml.sax.rst Outdated
and :class:`~xml.sax.xmlreader.XMLReader` interfaces are defined in the
module :mod:`xml.sax.xmlreader`. The handler interfaces are defined in
:mod:`xml.sax.handler`. For convenience,
:class:`~xml.sax.xmlreader.AttributesImpl`,

@StanFromIreland StanFromIreland Jul 2, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should instead link to :ref:`Attributes <attributes-objects>` and :ref:`AttributesNS <attributes-ns-objects>`.

@StanFromIreland StanFromIreland left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You didn't actually fix the issue with create_parser? Please change it to a reference as I noted before. You should then be able to remove the file from .nitignore.

Comment thread Doc/library/xml.sax.rst Outdated
:class:`~xml.sax.xmlreader.InputSource` (which is often
instantiated directly) and the handler classes are also available from
:mod:`!xml.sax`. These interfaces are described below.
:ref:`Attributes <attributes-objects>`,:ref:`AttributesNS <attributes-ns-objects>`,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing space here.

Comment thread Doc/library/xml.sax.rst Outdated
:mod:`!xml.sax`. These interfaces are described below.
:ref:`Attributes <attributes-objects>`,:ref:`AttributesNS <attributes-ns-objects>`,
and :class:`~xml.sax.xmlreader.XMLReader` interfaces are defined in the module
:mod:`xml.sax.xmlreader`. The handler interfaces are defined in

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep double spacing for consistency.

@manoj-marimuthu

Copy link
Copy Markdown
Contributor Author

@StanFromIreland I believe I have added the reference now, I am new to sphinx.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants